Skip to content

Conversation

@iamgabrielma
Copy link
Contributor

HACK week. Part of #15372

Description

This PR registers a new Keychain merchantAIProviderKey value for storing the associated merchant's AI API key, to be used in AI Settings when Jetpack AI is not used by default.

Simulator Screen Recording - iPhone 16 Plus - US store - 2025-09-17 at 14 05 56

Testing information

  • In AISettingsView .onAppear method, add:
debugPrint("\(viewModel.keychain)")
debugPrint("\(viewModel.keychain[WooConstants.merchantAIProviderKey])")
  • Enable allowMerchantAIAPIKey flag
  • Go to Menu > AI Settings.
  • Play around with the API field. Save, clear, update, get out, reload the view, etc, ..
  • Note how the key is stored/cleared/updated adequately
  • You'll also see the key and value printed in the console:
$ [\"key\": \"merchantAIProviderKey\", [ ... ]
$ "Optional(\"This is a test\")"

@iamgabrielma iamgabrielma added this to the 23.4 milestone Sep 17, 2025
@iamgabrielma iamgabrielma added type: task An internally driven task. feature: add/edit products Related to adding or editing products. Enhancement labels Sep 17, 2025
@iamgabrielma iamgabrielma marked this pull request as ready for review September 17, 2025 07:14
@wpmobilebot
Copy link
Collaborator

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr16136-f8e69a2
Version23.2
Bundle IDcom.automattic.alpha.woocommerce
Commitf8e69a2
Installation URL5bg5nedsh1guo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@iamgabrielma
Copy link
Contributor Author

Closing: pdfdoF-85K#comment-9421-p2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement feature: add/edit products Related to adding or editing products. type: task An internally driven task.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants